Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rename action types #222

Merged
merged 2 commits into from
May 17, 2016
Merged

rename action types #222

merged 2 commits into from
May 17, 2016

Conversation

abhiaiyer91
Copy link
Contributor

@abhiaiyer91 abhiaiyer91 commented May 16, 2016

Prefix Action Types with APOLLO_

#210

TODO:

  • Update CHANGELOG.md with your change
  • Make sure all of the significant new logic is covered by tests
  • Rebase your changes on master so that they can be merged easily
  • Make sure all tests and linter rules pass

@stubailo
Copy link
Contributor

Looks great!

@stubailo
Copy link
Contributor

stubailo commented May 16, 2016

Can you add to the change log please? And in the future it might be easier to keep the default checklist.

@abhiaiyer91
Copy link
Contributor Author

Will get to it when i am home! Where is the default checklist?

@stubailo
Copy link
Contributor

https://github.com/apollostack/apollo-client/blob/master/.github/PULL_REQUEST_TEMPLATE.md

It should show up as the default description when you make a new PR.

@abhiaiyer91
Copy link
Contributor Author

@stubailo All done thank you!

@stubailo
Copy link
Contributor

Looks good to me, let's merge after CI passes!

@stubailo stubailo merged commit 39633ab into master May 17, 2016
@stubailo
Copy link
Contributor

Published in 0.3.9!

@stubailo stubailo deleted the ActionTypes branch September 20, 2016 03:43
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants